Skip to content

fix(agents): preserve queued prompt across bounded context#530

Closed
HomenShum wants to merge 1 commit into
mainfrom
codex/fast-agent-empty-messages
Closed

fix(agents): preserve queued prompt across bounded context#530
HomenShum wants to merge 1 commit into
mainfrom
codex/fast-agent-empty-messages

Conversation

@HomenShum

Copy link
Copy Markdown
Owner

Summary

  • validate the exact queued user prompt before routing, reservation, or provider execution
  • pass that validated prompt explicitly while keeping provider history bounded at zero
  • fail closed on stale or malformed prompt rows
  • redact SDK error-class wrappers and map internal empty-prompt failures to a safe retry message

Production evidence

The first bounded production dogfood after #529 selected the correct tier-eligible model and released its reservation, but the SDK received an empty provider message list because recentMessages was zero and only promptMessageId was supplied.

Verification

  • focused FastAgent/runtime tests: 63/63
  • full FastAgentPanel suite: 213 passed, 19 skipped
  • app TypeScript: green
  • Convex TypeScript: green
  • production build: green
  • git diff --check: green
  • independent prompt-correctness review: approve
  • independent accounting/lease/cancellation review: approve
  • independent error-redaction review: approve

Release

P0 CI-gated auto-merge only. No admin bypass. No manual Convex production deploy.

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodebench-ai Building Building Preview, Comment Jul 14, 2026 4:38pm

Request Review

@HomenShum HomenShum enabled auto-merge (squash) July 14, 2026 16:38
@HomenShum HomenShum closed this Jul 14, 2026
auto-merge was automatically disabled July 14, 2026 16:39

Pull request was closed

@HomenShum HomenShum deleted the codex/fast-agent-empty-messages branch July 14, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants